Skip to content

Update MACOSX_DEPLOYMENT_TARGET 10.13 -> 10.15 #6511

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 8, 2025

Conversation

metascroy
Copy link
Contributor

@metascroy metascroy commented Apr 8, 2025

We are trying to bump the torchao pin in ExecuTorch, but CI jobs fail: pytorch/executorch#9947.

In particular this job fails: https://github.com/pytorch/executorch/actions/runs/14321828652/job/40140080161?pr=9947

The root cause is "/Users/runner/work/executorch/executorch/pytorch/executorch/third-party/ao/torchao/experimental/../../torchao/experimental/ops/memory.h:22:34: error: 'aligned_alloc' is only available on macOS 10.15 or newer [-Werror,-Wunguarded-availability-new]"

This is because the test infra defines the "export MACOSX_DEPLOYMENT_TARGET=10.13".

Copy link

vercel bot commented Apr 8, 2025

@metascroy is attempting to deploy a commit to the Meta Open Source Team on Vercel.

A member of the Team first needs to authorize it.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 8, 2025
@atalman
Copy link
Contributor

atalman commented Apr 8, 2025

hi @metascroy what is the test plan for these changes ?
Could you also please include some context in the PR description.

@metascroy
Copy link
Contributor Author

hi @metascroy what is the test plan for these changes ? Could you also please include some context in the PR description.

I added some context. How do you usually test these changes?

I see this similar PR from 2 years ago, but don't see tests attached to it (#4069).

Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@atalman atalman merged commit e0b305c into pytorch:main Apr 8, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants